Skip to content

SFX overhaul: real generated audio for the 12 silenced effects - #261

Merged
Vodkadav merged 1 commit into
mainfrom
feat/sfx-overhaul
Jul 17, 2026
Merged

SFX overhaul: real generated audio for the 12 silenced effects#261
Vodkadav merged 1 commit into
mainfrom
feat/sfx-overhaul

Conversation

@Vodkadav

Copy link
Copy Markdown
Owner

Replaces the deliberately-silenced placeholder SFX (tank-death explosion, wall break, generic pickup, all 9 per-powerup cues) with locally-generated Stable Audio clips (assetfactory MCP, CLAP-ranked best-of-N, ffmpeg trim/fade/highpass, peak-matched to fire.ogg). Every SfxKind is now mapped in SfxPool; the silent-placeholder path is gone.

  • New test EveryKind_ResolvesALoadedStream decodes all 20 clips in-engine (catches any non-vorbis regression)
  • Credits/provenance recorded in docs/credits/assets.md
  • Verified: build 0 errors; xUnit 14/69/552/54; headless GoDotTest 189 passed / 0 failed

Owner ear-check pending — clip quality is the one ungated review left; if any clip sounds off it can be regenerated individually.

🤖 Generated with Claude Code

Regenerate explosion, wall break, pickup and all nine powerup pickup cues
with Stable Audio Open via the assetfactory MCP (cartoonish one-shots,
1.0-2.0 s, trimmed + faded, 25 Hz highpass, peak-matched to fire.ogg,
OGG Vorbis q5). Rewire SfxFiles so every SfxKind maps to a real clip,
drop the silent-placeholder path, and add SfxPool.HasStream so the tests
can assert every kind decodes a real stream in-engine.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Vodkadav pushed a commit that referenced this pull request Jul 17, 2026
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@Vodkadav
Vodkadav merged commit 80df1cf into main Jul 17, 2026
8 checks passed
@Vodkadav
Vodkadav deleted the feat/sfx-overhaul branch July 17, 2026 07:25
Vodkadav added a commit that referenced this pull request Jul 17, 2026
* PROGRESS: record 2026-07-17 parallel batch (SFX overhaul, net victory screen, net teleport pads, perf check)

Three branches pushed pending owner merge + SFX ear-check; perf verdict recorded.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* PROGRESS: cite PRs #261/#262 for the 2026-07-17 batch

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

---------

Co-authored-by: Daniel Machado <au280502@uni.au.dk>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant